Skip to content

[overgeneral-exceptions] Only handle qualified names#8411

Merged
Pierre-Sassoulas merged 2 commits into
pylint-dev:mainfrom
Pierre-Sassoulas:overgeneral-exception-no-qname
Mar 9, 2023
Merged

[overgeneral-exceptions] Only handle qualified names#8411
Pierre-Sassoulas merged 2 commits into
pylint-dev:mainfrom
Pierre-Sassoulas:overgeneral-exception-no-qname

Conversation

@Pierre-Sassoulas
Copy link
Copy Markdown
Member

Type of Changes

Type
🔨 Refactoring
📜 Docs

Description

I'm thinking of creating a pylint "check configuration" script, to avoid checking that at run time. Let me know what you think ;)

@Pierre-Sassoulas Pierre-Sassoulas added Breaking changes for 3.0 🦤 Maintenance Discussion or action around maintaining pylint or the dev workflow labels Mar 9, 2023
@Pierre-Sassoulas Pierre-Sassoulas added this to the 3.0.0a6 milestone Mar 9, 2023
Comment thread doc/whatsnew/fragments/8410.user_action Outdated
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 9, 2023

Codecov Report

Merging #8411 (a05bc74) into main (b2ab82a) will not change coverage.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #8411   +/-   ##
=======================================
  Coverage   95.68%   95.68%           
=======================================
  Files         175      175           
  Lines       18465    18465           
=======================================
  Hits        17668    17668           
  Misses        797      797           
Impacted Files Coverage Δ
pylint/checkers/exceptions.py 97.85% <100.00%> (ø)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Mar 9, 2023

🤖 According to the primer, this change has no effect on the checked open source code. 🤖🎉

This comment was generated for commit a05bc74

@ollie-iterators
Copy link
Copy Markdown
Contributor

Type of Changes

Type
✓ 🔨 Refactoring
✓ 📜 Docs

Description

I'm thinking of creating a pylint "check configuration" script, to avoid checking that at run time. Let me know what you think ;)

That sounds like a good change to do.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Breaking changes for 3.0 🦤 Maintenance Discussion or action around maintaining pylint or the dev workflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants